Skip to content

parameterize token secret per organization to prevent decrypt errors - #19

Merged
stefan-as merged 1 commit into
mainfrom
fix/8122
Apr 10, 2025
Merged

parameterize token secret per organization to prevent decrypt errors#19
stefan-as merged 1 commit into
mainfrom
fix/8122

Conversation

@stefan-as

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread src/settings.js
Comment on lines +9 to +11
t.storeSecret(tokenName, window.togglToken.value).then(() => {
t.closePopup();
});

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ich frage mich hier, ob wir für den fall, dass storeSecret nicht erfolgreich ist, nicht wenigstens ein console.log haben wollen.

@aboutnick aboutnick left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so, alles schön und gut. nur leider kann ich nirgendwo nachvollziehen, warum das jetzt die lösung für markus' problem sein sollte. daher denke ich, die idee ist, hier schnell voranzugehen und es einfach auszuprobieren.

🚀 🎸

@stefan-as

Copy link
Copy Markdown
Contributor Author

@aboutnick Das Problem entsteht, weil verschiedene Orgas mit unterschiedlichen Schlüsseln auf dasselbe Secret zugreifen. Deswegen sorgt diese Änderung dafür, dass das ein Secret pro Orga wird und nicht mehr ein einziges Secret für alle.

@stefan-as
stefan-as merged commit 43d23c7 into main Apr 10, 2025
@stefan-as
stefan-as deleted the fix/8122 branch April 10, 2025 08:05
@aboutnick

Copy link
Copy Markdown

weil verschiedene Orgas

@stefan-as da gehts schon los. was für verschiedene orgas? mir ist das gesamte scenario unklar. evtl schnattern wir mal, wenn du mal kurz 10 min luft hast. zb an einem freitag.. 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants